Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: fix incorrect version notation of GQL Field #1993

Merged
merged 41 commits into from
May 27, 2024

Conversation

fregataa
Copy link
Member

@fregataa fregataa commented Apr 3, 2024

blocked by #2000

Change

  • Correct Added/Deprecated version of Resource policy GQL schema fields
  • Unify description/deprecation message to Added in {VERSION} and Deprecated since {VERSION} for consistency.
  • Unify version notation to end with .(dot). e.g. "23.09.10" -> "23.09.10."
  • Fix fields labeled as added in version "24.03.1" that were actually added in version "24.03.0"

Why are the versions so different?

KeyPairResourcePolicy

UserResourcePolicy, ProjectResourcePolicy

{Create/Modify}UserResourcePolicyInput, ProjectResourcePolicyInput

Warning

in 23.09, max_vfolder_count field of {User/Project}ResourcePolicy was removed in "23.09.3." by mistake and restored in "23.09.6.". So, the version the field was added in is "23.09.6."

  • The commit that removed max_vfolder_count from {User/Project}ResourcePolicy: f4d6b51
  • The commit that restored the field: 3d9a886

Some fields has a different version in each major version. e.g. {User/Project}ResourcePolicy has max_vfolder_count field and it was added in "24.03.1" and "23.09.6".

Checklist: (if applicable)

  • Milestone metadata specifying the target backport version
  • API server-client counterparts (e.g., manager API -> client SDK)
  • Documentation
    • Contents in the docs directory
    • docstrings in public interfaces and type annotations

📚 Documentation preview 📚: https://sorna--1993.org.readthedocs.build/en/1993/


📚 Documentation preview 📚: https://sorna-ko--1993.org.readthedocs.build/ko/1993/

@fregataa fregataa added this to the 23.09 milestone Apr 3, 2024
@fregataa fregataa self-assigned this Apr 3, 2024
@github-actions github-actions bot added comp:manager Related to Manager component size:M 30~100 LoC labels Apr 3, 2024
@kyujin-cho kyujin-cho changed the title chore: Correct a version notation of Resource policy GQL schema chore: fix incorrect version notation of GQL Field Apr 4, 2024
@github-actions github-actions bot added size:L 100~500 LoC and removed size:M 30~100 LoC labels Apr 4, 2024
@fregataa fregataa added the urgency:blocker IT SHOULD BE RESOLVED BEFORE NEXT RELEASE! label Apr 4, 2024
@fregataa fregataa marked this pull request as ready for review May 25, 2024 04:35
@kyujin-cho kyujin-cho added area:docs Documentations and removed area:docs Documentations labels May 27, 2024 — with Graphite App
@fregataa fregataa requested a review from agatha197 May 27, 2024 05:35
Copy link
Contributor

@agatha197 agatha197 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fregataa fregataa merged commit 24e1e29 into main May 27, 2024
27 of 28 checks passed
@fregataa fregataa deleted the chore/clear-resource-policy-field-version branch May 27, 2024 06:32
lablup-octodog pushed a commit that referenced this pull request May 27, 2024
Co-authored-by: Sion Kang <siontama@gmail.com>
Backported-from: main (24.09)
Backported-to: 24.03
Backport-of: 1993
Copy link

Backport to 23.09 is failed. Please backport manually.

fregataa added a commit that referenced this pull request May 27, 2024
Backported-from: main (24.09)
Backported-to: 23.09
Co-authored-by: Sion Kang <siontama@gmail.com>
yomybaby pushed a commit to lablup/backend.ai-webui that referenced this pull request May 28, 2024
follows #2357 
Add User Resource Policy Page

## Description
- Add User resource policy CRUD.
- Add `FormItemProps` props to `FormItemWithUnlimited`.
- Add simple error handler to `KeypairResourcePolicySettingModal`.
- Remove unused component, `ResourcePolicyCard`.

## Checklist
- [ ] List Page: name, max folder count, max session count per model session, max quota scope size, id, created at
  - [ ] Each item will be shown/hidden according to the server version. 
  - [ ] In particular, it is stated in the schema that `max_vfolder_count` is supported from version 24.03.1, but this is incorrectly documented, and it follows lablup/backend.ai#1993.
  - [ ] `max_session_count_per_model_session` is supported from version 23.09.10 as follows lablup/backend.ai#1993 too.
- [ ] Create/Modal: Create or modify user resource policy.
  - [ ] Version compatibility is the same as the list.
  - [ ] Unlimited value of `Max Folder Count` is 0.
  - [ ] Unlimited value of `Max Folder Size` is -1.

## Screenshots
<table>
<tr>
 <td>
 <td>24.03.0b1 (10.82.230.101)</td>
 <td>24.09.0dev1 (main)</td>
<tr>
 <td>List</td>
 <td>

![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/2HueYSdFvL8pOB5mgrUQ/7c23ba67-e22f-4770-8a4a-dfef8856ba96.png)

</td>
 <td>

![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/2HueYSdFvL8pOB5mgrUQ/c42974ac-fa28-4abb-a7a9-e2c6f67af335.png)

</td>
<tr>
 <td>Create Modal</td>
 <td>

![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/2HueYSdFvL8pOB5mgrUQ/da4d4c9d-a5cb-48fc-88c2-ca801642fde7.png)

</td>
 <td>

![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/2HueYSdFvL8pOB5mgrUQ/887306a4-00b9-4758-a72f-df897c60d634.png)

</td>
<tr>
 <td>Modify Modal</td>
 <td>

![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/2HueYSdFvL8pOB5mgrUQ/88c9f6e7-26fd-4780-b9fd-ce242724fcf1.png)

</td>
 <td>

![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/2HueYSdFvL8pOB5mgrUQ/af6c825c-5fb1-4417-a21c-f946e0f9ad3a.png)

</td>
</tr>
<tr>
<td>Table Setting</td>
<td>

![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/2HueYSdFvL8pOB5mgrUQ/5a6dc107-9ff2-4b14-87d2-badd419b242d.png)

</td>
<td>

![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/2HueYSdFvL8pOB5mgrUQ/ec04ff59-5f3f-47a8-a3a5-4b192c5528c5.png)

</td>
</tr>
</table>

**Checklist:** (if applicable)

- [x] Mention to the original issue
- [ ] Documentation
- [x] Minium required manager version
- [x] Specific setting for review (eg., KB link, endpoint or how to setup)
- [x] Minimum requirements to check during review
- [ ] Test case(s) to demonstrate the difference of before/after
yomybaby pushed a commit to lablup/backend.ai-webui that referenced this pull request May 28, 2024
follows #2357 
Add Project Resource Policy Page


### TL;DR
This pull request introduces changes related to FormItemWithUnlimited and ProjectResourcePolicyList components.

### What changed?
The changes include:
- Addition of FormItemWithUnlimitedProps in `FormItemWithUnlimited` component
- Updates to `ProjectResourcePolicyList` component with enhanced functionality and UI improvements

### Checklist
- [ ] List Page: name, max folder count, max quota scope size, id, created at
  - [ ] Each item will be shown/hidden according to the server version. 
  - [ ] In particular, it is stated in the schema that `max_vfolder_count` is supported from version 24.03.1, but this is incorrectly documented, and it follows lablup/backend.ai#1993.
- [ ] Create/Modal: Create or modify project resource policy.
  - [ ] Version compatibility is the same as the list.
  - [ ] Unlimited value of `Max Folder Count` is 0.
  - [ ] Unlimited value of `Max Folder Size` is -1.


### Screenshots
<table>
<tr>
<td></td>
 <td>24.03.0b1 (10.82.230.101)</td>
 <td>24.09.0dev1 (main)</td>
</tr>
<tr>
<td>List</td>
<td>

![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/2HueYSdFvL8pOB5mgrUQ/bc9a3d28-9119-47f7-b697-f1d54935c020.png)


</td>
<td>

![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/2HueYSdFvL8pOB5mgrUQ/21e03b17-1d18-4fb7-8683-7bcdfe009beb.png)


</td>
</tr>
<tr>
<td>Create Modal</td>
<td>

![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/2HueYSdFvL8pOB5mgrUQ/93f5f421-3c09-47f6-ab7e-9404c4bdbb8c.png)


</td>
<td>

![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/2HueYSdFvL8pOB5mgrUQ/8c958f14-b3e7-427d-997c-aa5a331343a2.png)

</td>
</tr>
<tr>
<td>Modify Modal</td>
<td>

![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/2HueYSdFvL8pOB5mgrUQ/81144f9b-a55a-4136-b298-58038f2f224b.png)


</td>
<td>

![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/2HueYSdFvL8pOB5mgrUQ/32d1f8d6-9fa9-4377-aee0-0427183fa573.png)


</td>
</tr>
<tr>
<td>Table Setting</td>
<td>

![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/2HueYSdFvL8pOB5mgrUQ/fc7e605b-b957-47fd-95ec-c92381931d0a.png)

</td>
<td>

![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/2HueYSdFvL8pOB5mgrUQ/11659c5b-cb04-4a5f-97fc-d1122913e1a5.png)

</td>
</tr>
</table>

---

<!--
Please precisely, concisely, and concretely describe what this PR changes, the rationale behind codes,
and how it affects the users and other developers.
-->

**Checklist:** (if applicable)

- [x] Mention to the original issue
- [ ] Documentation
- [x] Minium required manager version
- [x] Specific setting for review (eg., KB link, endpoint or how to setup)
- [x] Minimum requirements to check during review
- [ ] Test case(s) to demonstrate the difference of before/after
kyujin-cho pushed a commit that referenced this pull request May 28, 2024
Co-authored-by: Sanghun Lee <sanghun@lablup.com>
Co-authored-by: Sion Kang <siontama@gmail.com>
Backported-from: main (24.09)
Backported-to: 24.03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:docs Documentations comp:manager Related to Manager component size:L 100~500 LoC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants